Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chainwriter tests #13360

Merged
merged 7 commits into from
May 30, 2024
Merged

Chainwriter tests #13360

merged 7 commits into from
May 30, 2024

Conversation

silaslenihan
Copy link
Contributor

No description provided.

@silaslenihan silaslenihan changed the title Started relayer evm tests Chainwriter tests May 29, 2024
@silaslenihan silaslenihan marked this pull request as ready for review May 29, 2024 20:34
@silaslenihan silaslenihan requested a review from a team as a code owner May 29, 2024 20:34
@silaslenihan silaslenihan requested review from archseer and nickcorin and removed request for a team May 29, 2024 20:34
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

@archseer archseer merged commit 5c2076d into cap-idempotent-writes May 30, 2024
58 of 73 checks passed
@archseer archseer deleted the chainwriter-tests branch May 30, 2024 08:15
archseer pushed a commit that referenced this pull request May 31, 2024
* Started relayer evm tests

* Evm Relay tests

* Added generic tests and additional evm tests for WriteTarget

* added changeset

* Update real-tools-tap.md

* lint

* small fix
github-merge-queue bot pushed a commit that referenced this pull request May 31, 2024
* evm: Add a stub chainwriter impl

* evm: Fix config parameter

* evm: Change the chainwriter receiver name

* evm: Remove the chain writer interface to reference chainlink-common

* evm: Update common dep, and fix signature

* go.sum: Run gomodtidy

* .changeset: Add a changeset

* evm: Pseudo-implement the submit transaction method on chainwriter

* evm: Add txm dependency to chainwriter

* evm: Use the txm param properly

* Update code to use the new interface

* nix: use monthly foundry branch that's persistent

* capabilities: Add config validation to write_target

* capabilities: Pass context into InitializeWrite

* minor: Resolve some inapplicable TODOs

* capabilities: Refactor write target by extracting commmon bits

* Refactor WriteTarget to use ChainWriter

* capabilities: Move evm specific code inside the relayer

* Chainwriter tests (#13360)

* Started relayer evm tests

* Evm Relay tests

* Added generic tests and additional evm tests for WriteTarget

* added changeset

* Update real-tools-tap.md

* lint

* small fix

* Update chainlink-common to include the new interface

* write_target: Fix tests

* Move target capability init inside evm.NewRelayer()

* go mod tidy

* Address lints

* scripts: go mod tidy

* integration-tests: go mod tidy

* evm: Only initialize write target if config actually present

* more tidy, fix last lint

* evm: Move send strategy to be config driven

* evm: Add some todos back

* .changeset: Add a changeset

* write_target: Mock out Config before NewRelayer is called

* Regenerate mocks with right version

* fix evm tests

* goimports

* fix evm tests

---------

Co-authored-by: Nick Corin <[email protected]>
Co-authored-by: Silas Lenihan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants